Tweak escaping on Windows shells

Also detect if we're on Windows whether we're an MSYS or cmd.exe shell at
runtime

diff --git a/src/cargo/util/shell_escape.rs b/src/cargo/util/shell_escape.rs
index f2694254c5d50de7bdfcb614a081c81412979e94..2acbd00ea004b4ca8ddba723ccebcd08069f4d99 100644 (file)
--- a/src/cargo/util/shell_escape.rs
+++ b/src/cargo/util/shell_escape.rs
@@ -9,10 +9,13 @@
 // except according to those terms.
 
 use std::borrow::Cow;
+use std::env;
 
 pub fn escape(s: Cow<str>) -> Cow<str> {
     if cfg!(unix) {
         unix::escape(s)
+    } else if env::var("MSYSTEM").is_ok() {
+        unix::escape(s)
     } else {
         windows::escape(s)
     }
@@ -20,50 +23,50 @@ pub fn escape(s: Cow<str>) -> Cow<str> {
 
 pub mod windows {
     use std::borrow::Cow;
+    use std::iter::repeat;
 
-    const SPACE: char = ' ';
-    const ESCAPE_CHAR: char = '\\';
-    const QUOTE_CHAR: char = '"';
-    const BACKSLASH: char = '\\';
-    const SLASH: char = '/';
-
-    /// Escape characters that may have special meaning in a shell,
-    /// including spaces.
+    /// Escape for the windows cmd.exe shell, for more info see this url:
     ///
-    /// Escape double quotes and spaces by wrapping the string in double quotes.
-    ///
-    /// Turn all backslashes into forward slashes.
+    /// http://blogs.msdn.com/b/twistylittlepassagesallalike/archive/2011/04/23
+    ///                      /everyone-quotes-arguments-the-wrong-way.aspx
     pub fn escape(s: Cow<str>) -> Cow<str> {
-        // check if string needs to be escaped
-        let mut has_spaces = false;
-        let mut has_backslashes = false;
-        let mut has_quotes = false;
+        let mut needs_escape = false;
         for ch in s.chars() {
             match ch {
-                QUOTE_CHAR => has_quotes = true,
-                SPACE => has_spaces = true,
-                BACKSLASH => has_backslashes = true,
+                '"' | '\t' | '\n' | ' ' => needs_escape = true,
                 _ => {}
             }
         }
-        if !has_spaces && !has_backslashes && !has_quotes {
+        if !needs_escape {
             return s
         }
         let mut es = String::with_capacity(s.len());
-        if has_spaces {
-            es.push(QUOTE_CHAR);
-        }
-        for ch in s.chars() {
-            match ch {
-                BACKSLASH => { es.push(SLASH); continue }
-                QUOTE_CHAR => es.push(ESCAPE_CHAR),
-                _ => {}
+        es.push('"');
+        let mut chars = s.chars().peekable();
+        loop {
+            let mut nslashes = 0;
+            while let Some(&'\\') = chars.peek() {
+                chars.next();
+                nslashes += 1;
             }
-            es.push(ch)
-        }
-        if has_spaces {
-            es.push(QUOTE_CHAR);
+
+            match chars.next() {
+                Some('"') => {
+                    es.extend(repeat('\\').take(nslashes * 2 + 1));
+                    es.push('"');
+                }
+                Some(c) => {
+                    es.extend(repeat('\\').take(nslashes));
+                    es.push(c);
+                }
+                None => {
+                    es.extend(repeat('\\').take(nslashes * 2));
+                    break;
+                }
+            }
+
         }
+        es.push('"');
         es.into()
     }
 
@@ -73,9 +76,9 @@ pub mod windows {
         assert_eq!(escape("linker=gcc -L/foo -Wl,bar".into()),
                           r#""linker=gcc -L/foo -Wl,bar""#);
         assert_eq!(escape(r#"--features="default""#.into()),
-                          r#"--features=\"default\""#);
+                          r#""--features=\"default\"""#);
         assert_eq!(escape(r#"\path\to\my documents\"#.into()),
-                          r#""/path/to/my documents/""#);
+                          r#""\path\to\my documents\\""#);
     }
 }
 

diff --git a/tests/support/mod.rs b/tests/support/mod.rs
index 6b013eef27db57bebf3175f34befefa626d0504d..24cda579f77e25703be46d0c8b7db26fa46d7a81 100644 (file)
--- a/tests/support/mod.rs
+++ b/tests/support/mod.rs
@@ -136,7 +136,8 @@ impl ProjectBuilder {
         let mut p = process(program).unwrap();
         p.cwd(&self.root())
          .env("HOME", &paths::home())
-         .env_remove("CARGO_HOME");
+         .env_remove("CARGO_HOME")  // make sure we don't pick up an outer one
+         .env_remove("MSYSTEM");    // assume cmd.exe everywhere on windows
         return p;
     }